Skip to content

Conversation

hsubramony
Copy link
Contributor

port nixl

@hsubramony
Copy link
Contributor Author

hsubramony commented Aug 30, 2025

the latest commit 9322571 works with vllm commit 17373dcd93ca60554d72cef4e159e70abbfd15af

@hsubramony
Copy link
Contributor Author

updated to latest vllm-gaudi + latest vllm
need nixl==0.4.1 to work

@hsubramony hsubramony marked this pull request as ready for review September 3, 2025 20:14
@hsubramony hsubramony changed the title initial port initial port for nixl Sep 3, 2025
@xuechendi
Copy link
Collaborator

@hsubramony , please fix pre-commit formatting issues

@xuechendi
Copy link
Collaborator

@hsubramony , please fix for failed CI and pre_commit

Signed-off-by: Harish Subramony <[email protected]>
Signed-off-by: Harish Subramony <[email protected]>
Signed-off-by: Harish Subramony <[email protected]>
Signed-off-by: Harish Subramony <[email protected]>
Signed-off-by: Harish Subramony <[email protected]>
Signed-off-by: Harish Subramony <[email protected]>
Signed-off-by: Harish Subramony <[email protected]>
Signed-off-by: Harish Subramony <[email protected]>
Signed-off-by: Harish Subramony <[email protected]>
Signed-off-by: Chendi.Xue <[email protected]>
Signed-off-by: Harish Subramony <[email protected]>
Signed-off-by: Harish Subramony <[email protected]>
@xuechendi
Copy link
Collaborator

Did local test with sharegpt perf evaluation on both G2 and G3 [Regular path], see no perf regression

Signed-off-by: Harish Subramony <[email protected]>
@xuechendi xuechendi merged commit dc0d049 into vllm-project:main Sep 11, 2025
7 checks passed
kfojcik-intel pushed a commit to kfojcik-intel/vllm-gaudi that referenced this pull request Sep 12, 2025
port nixl

---------

Signed-off-by: Harish Subramony <[email protected]>
Signed-off-by: Chendi.Xue <[email protected]>
Co-authored-by: Chendi.Xue <[email protected]>
Signed-off-by: Katarzyna Fojcik <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants